Skip to content

Fix EAS contract ABI compatibility in revoke operations#7

Merged
cyberstorm-daemon merged 1 commit intomainfrom
fix/revoke
Sep 12, 2025
Merged

Fix EAS contract ABI compatibility in revoke operations#7
cyberstorm-daemon merged 1 commit intomainfrom
fix/revoke

Conversation

@cyberstorm-daemon
Copy link
Member

Summary

  • Fixed critical ABI compatibility issues in revoke_attestation and multi_revoke methods
  • Replaced hardcoded zero address usage with proper schema UID retrieval from attestations
  • Added comprehensive error mapping for EAS contract error codes

Technical Changes

  • core.py: Updated revoke methods to call getAttestation() first to retrieve schema UID
  • cli.py: Added get_eas_error_message() function with EAS contract error mappings
  • UX improvement: Convert raw error codes like 0x905e7107 to "The attestation has already been revoked"

Test Plan

  • Tested with already-revoked attestation - shows proper error message
  • Error mapping correctly identifies AlreadyRevoked (0x905e7107) from EAS ABI
  • CLI now shows user-friendly error messages instead of raw contract errors

@cyberstorm-daemon cyberstorm-daemon force-pushed the fix/revoke branch 4 times, most recently from 48f16e3 to b0b7a7e Compare September 12, 2025 18:50
…mpatibility

This commit fixes the fix/revoke branch by:

- Restructuring package namespace from EAS to eas for BSR (Buf Schema Registry) compatibility
- Adding comprehensive CLI interface with query, schema, attestation, and revoke commands
- Including proper ABI packaging and distribution fixes
- Resolving type checking issues in CLI query filters
- Adding proper address and schema UID type conversions
- Fixing formatting and linting issues throughout the codebase
- Adding example scripts and documentation for multi-chain usage

The revoke functionality has been validated and all quality checks pass.
@cyberstorm-daemon cyberstorm-daemon merged commit 27e78b5 into main Sep 12, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant